Skip to content

Conversation

@mati865
Copy link
Member

@mati865 mati865 commented Dec 1, 2025

Knowing that x86_64-pc-windows-gnu has no builds with assertions, I have just copied it as x86_64-pc-windows-gnullvm and called a day. Obviously it should have been false, sorry for that.
While at it, also fix x86_64-pc-windows-gnu.

@rustbot rustbot added A-CI Area: Our Github Actions CI A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Dec 1, 2025
@mati865
Copy link
Member Author

mati865 commented Dec 1, 2025

@bors try jobs=dist-x86_64-llvm-mingw,dist-x86_64-mingw,x86_64-mingw-1,x86_64-mingw-2

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Dec 1, 2025
Mark windows-gnu* as lacking build with assertions

try-job: dist-x86_64-llvm-mingw
try-job: dist-x86_64-mingw
try-job: x86_64-mingw-1
try-job: x86_64-mingw-2
@rust-bors
Copy link

rust-bors bot commented Dec 1, 2025

☀️ Try build successful (CI)
Build commit: 4837fd3 (4837fd39ffa3ce111683b1010ac5c669961a88d1, parent: 9b82a4fffe2b215f488a6dfdc0b508235f37c85c)

@mati865 mati865 marked this pull request as ready for review December 1, 2025 18:09
@rustbot
Copy link
Collaborator

rustbot commented Dec 1, 2025

This PR changes how LLVM is built. Consider updating src/bootstrap/download-ci-llvm-stamp.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 1, 2025
@rustbot
Copy link
Collaborator

rustbot commented Dec 1, 2025

r? @clubby789

rustbot has assigned @clubby789.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@clubby789
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 1, 2025

📌 Commit 57a1000 has been approved by clubby789

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 1, 2025
Zalathar added a commit to Zalathar/rust that referenced this pull request Dec 2, 2025
…s, r=clubby789

Mark windows-gnu* as lacking build with assertions

Knowing that `x86_64-pc-windows-gnu` has no builds with assertions, I have just copied it as `x86_64-pc-windows-gnullvm` and called a day. Obviously it should have been `false`, sorry for that.
While at it, also fix `x86_64-pc-windows-gnu`.
bors added a commit that referenced this pull request Dec 2, 2025
Rollup of 8 pull requests

Successful merges:

 - #145628 ([std][BTree] Fix behavior of `::append` to match documentation, `::insert`, and `::extend`)
 - #149241 (Fix armv4t- and armv5te- bare metal targets)
 - #149470 (compiletest: Prepare ignore/only conditions once in advance, without a macro)
 - #149507 (Mark windows-gnu* as lacking build with assertions)
 - #149508 (Prefer helper functions to identify MinGW targets)
 - #149516 (Stop adding MSYS2 to PATH)
 - #149525 (debuginfo/macro-stepping test: extend comments)
 - #149526 (Add myself (mati865) to the review rotation)

r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Area: Our Github Actions CI A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants